Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add arguments #180

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

feat: Add arguments #180

wants to merge 4 commits into from

Conversation

engeir
Copy link

@engeir engeir commented Jan 10, 2024

This PR adds an option to pass any number of arguments to the latexmk command. Effectively this would also make the format option redundant.

I haven't done any testing, other than using it for one project as:

...
      - name: Generate PDF document
        # uses: hspaans/latexmk-action@v1
        uses: engeir/latexmk-action@add-arguments
        with:
          format: pdf
          filename: src/thesis.tex
          arguments: |
            -cd
            -output-directory="../build/"
...

A check for the -pvc argument is included, as the action would not finish otherwise. Not sure if more arguments should be restricted or if this is enough.

@hspaans hspaans mentioned this pull request Aug 5, 2024
@hspaans
Copy link
Owner

hspaans commented Aug 5, 2024

The idea/code has been merged in https://github.com/hspaans/latexmk-action/releases/tag/v2.0.0 for more testing as part of a pre-release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants